Archive legacy Modernization Scanner guidance - #10959
Conversation
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
|
||
| - [Connect a classic SharePoint team site to a new Microsoft 365 group](../../transform/modernize-connect-to-office365-group.md) | ||
| - [SharePoint Modernization scanner tool](https://github.com/SharePoint/sp-dev-modernization/tree/master/Tools/SharePoint.Modernization) | ||
| - [Legacy SharePoint Modernization Scanner](../../transform/modernize-scanner.md) |
There was a problem hiding this comment.
to PM: do we need legacy prefix?
to copilot: the path is correct. don't modify.
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit a74ba66: 💡 Validation status: suggestions
This comment lists only the first 25 files in the pull request. docs/community/open-source-projects.md
docs/transform/modernize-scanner-faq.yml
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit a7d8f2c: 💡 Validation status: suggestions
This comment lists only the first 25 files in the pull request. docs/community/open-source-projects.md
docs/transform/modernize-scanner-faq.yml
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. |
a7d8f2c to
baa56b9
Compare
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit baa56b9: 💡 Validation status: suggestions
This comment lists only the first 25 files in the pull request. docs/community/open-source-projects.md
docs/transform/modernize-scanner-faq.yml
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit d21ec2d: 💡 Validation status: suggestions
This comment lists only the first 25 files in the pull request. docs/community/open-source-projects.md
docs/transform/modernize-scanner-faq.yml
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit d6278a1: 💡 Validation status: suggestions
This comment lists only the first 25 files in the pull request. docs/community/open-source-projects.md
docs/sp-add-ins/retirement-announcement-for-azure-acs.md
docs/transform/modernize-scanner-faq.yml
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. |
d6278a1 to
46132b5
Compare
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 46132b5: 💡 Validation status: suggestions
This comment lists only the first 25 files in the pull request. docs/community/open-source-projects.md
docs/sp-add-ins/retirement-announcement-for-azure-acs.md
docs/transform/modernize-scanner-faq.yml
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. |
Andrew Connell (andrewconnell)
left a comment
There was a problem hiding this comment.
revert all changes to ms.date; this is the date the article was published, not updated
I only called this out in the first file, but this must be revertted in all.
| title: Guidance - Migrate from classic workflows to Power Automate flows in SharePoint | ||
| description: This article specifically provides guidance about how to plan for transitioning from classic SharePoint Workflows to Power Automate flows. | ||
| ms.date: 05/12/2023 | ||
| ms.date: 07/31/2026 |
There was a problem hiding this comment.
revert this change... ms.date is the date the article was published
There was a problem hiding this comment.
Reverted this ms.date change and all 28 other ms.date changes in commit 81e3f8530. The PR now has no ms.date diff against main; OpenPublishing, PoliCheck, and CLA pass.
46132b5 to
424ada1
Compare
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 424ada1: 💡 Validation status: suggestions
This comment lists only the first 25 files in the pull request. docs/community/open-source-projects.md
docs/sp-add-ins/retirement-announcement-for-azure-acs.md
docs/transform/modernize-scanner-faq.yml
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 89b7e81: 💡 Validation status: suggestions
This comment lists only the first 25 files in the pull request. docs/community/open-source-projects.md
docs/sp-add-ins/retirement-announcement-for-azure-acs.md
docs/transform/modernize-scanner-faq.yml
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. |
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 81e3f85: 💡 Validation status: suggestions
This comment lists only the first 25 files in the pull request. docs/community/open-source-projects.md
docs/sp-add-ins/retirement-announcement-for-azure-acs.md
docs/transform/modernize-scanner-faq.yml
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. |
|
Andrew Connell (@andrewconnell) Addressed the requested changes in |
|
TaoziZ03 sorry I missed this in my last review, but please fix the two absolute link findings in the automated build check:
|
Make Microsoft 365 Assessment the active modernization assessment entry, retain the legacy Scanner documentation under Archive, and update scenario references according to current capability coverage. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144
Keep the existing SharePoint/sp-dev-modernization URLs because GitHub redirects them to the canonical pnp/modernization repository. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144
Restore the original dates on files whose link changes were reverted, keeping the PR focused on meaningful content updates. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144
Keep direct PnP Assessment references within the Learn Assessment articles and preserve current workflow retirement guidance. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144
Keep the active assessment and transformation workflow ahead of the archived documentation in the root navigation. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144
Update only the root TOC label while preserving the landing page title, file name, and Learn URL. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144
Revert every ms.date edit requested in review while preserving the documentation and navigation changes. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144
Replace the two absolute Microsoft Learn targets identified by OpenPublishing with root-relative cross-docset paths. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144
81e3f85 to
954498f
Compare
PoliCheck Scan ReportThe following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans. ✅ No issues foundMore information about PoliCheckInformation: PoliCheck | Severity Guidance | Term |
|
Learn Build status updates of commit 954498f: 💡 Validation status: suggestions
This comment lists only the first 25 files in the pull request. docs/community/open-source-projects.md
docs/transform/modernize-scanner-faq.yml
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. |
|
Andrew Connell (@andrewconnell) Fixed both |
fe38473
into
SharePoint:main
* Docs: align packaging and setup articles with Heft (SPFx 1.22) (#10962) * Update SPFx docs for Heft toolchain terminology. Align packaging and setup articles with SPFx v1.22 Heft commands so readers are not directed to legacy gulp tasks. * Revert ms.date changes on updated SPFx docs Keep the original publish dates; only the Heft wording updates stay. * Clarify supported Entra service principal types for SharePoint Embedded (#10965) * Clarified service principle types support for SPE * Update ms.date for authentication and authorization doc * doc(): address msdocs auto review findings --------- Co-authored-by: Andrew Connell <me@andrewconnell.com> * Update SPE docs for retrieval API 1.0 with PAYG (#10967) * Update SPE docs for retrieval API 1.0 with PAYG * Address PR feedback on retrieval API docs - Restore original ms.date on all articles; never bump on existing docs - Present Foundry knowledge source and Retrieval API as two separate options, and note the Retrieval API works inside a Foundry agent - Combine the request verb, header, and body into one http block - Remove resourceMetadata from samples; it takes fields like title or author, not containerTypeId - Use a generic webUrl placeholder and treat the URL as opaque because urlTemplate changes its shape - Fix the broken Retrieval API reference link Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Clarify SPE data source for retrieval is in preview --------- Co-authored-by: pemtaira <pemtaira+odspmdb@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * Add 'Copilot' to commandBarCustomization keys (#10969) * Fix reference code for SPE (#10974) Co-authored-by: dilucesr <dilucesr+odspmdb@microsoft.com> * Restore modernization article publication dates (#10979) * Restore modernization article publication dates Restore ms.date values changed by the Stage 2 documentation PR while retaining publication dates for newly introduced articles. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144 * docs(transform): 📝 reset ms.date to publish dates - set ms.date from oldest commit touching each file - assessment tool articles: 07/27/2026 -> 07/22/2026 - modernize-userinterface-site-pages-powershell: 02/09/2023 -> 11/19/2018 - modernize-userinterface-site-pages: 08/13/2024 -> 03/22/2018 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> --------- Co-authored-by: titao <titao+odspmdb@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Andrew Connell <me@andrewconnell.com> Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144 * Add notes about SharePoint Workflows deprecation schedules (#10978) * Add notes about SharePoint Workflows deprecation schedules Both on-prem and online * docs: update wording and tense for SharePoint workflows retirement Updated article to reflect the retirement of SharePoint 2010 and 2013 workflows, and clarified the authoring environments for SharePoint workflows. * docs: update wording and tense for SharePoint workflows retirement Updated article to reflect the retirement of SharePoint 2010 and 2013 workflows, and clarified the authoring environments for SharePoint workflows. --------- Co-authored-by: Andrew Connell <me@andrewconnell.com> * Archive legacy Modernization Scanner guidance (#10959) * Archive legacy modernization scanner guidance Make Microsoft 365 Assessment the active modernization assessment entry, retain the legacy Scanner documentation under Archive, and update scenario references according to current capability coverage. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144 * Preserve legacy modernization repository links Keep the existing SharePoint/sp-dev-modernization URLs because GitHub redirects them to the canonical pnp/modernization repository. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144 * Remove date-only documentation changes Restore the original dates on files whose link changes were reverted, keeping the PR focused on meaningful content updates. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144 * Make Microsoft Learn the primary assessment entry Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144 * Route assessment links through Microsoft Learn Keep direct PnP Assessment references within the Learn Assessment articles and preserve current workflow retirement guidance. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144 * Place classic page workflow before Archive Keep the active assessment and transformation workflow ahead of the archived documentation in the root navigation. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144 * Rename classic page modernization navigation Update only the root TOC label while preserving the landing page title, file name, and Learn URL. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144 * Restore original article dates Revert every ms.date edit requested in review while preserving the documentation and navigation changes. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144 * Use relative Microsoft Learn links Replace the two absolute Microsoft Learn targets identified by OpenPublishing with root-relative cross-docset paths. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144 --------- Co-authored-by: titao <titao+odspmdb@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144 * Removed 100K container limit and added limit for custom columns per container (#10981) Added a new limit for custom columns per container to be 40. Also removed the 100K container limit. There is no hard upper limit and it confuses customers. * RateLimit guidance for SharePoint Online (#10983) Updating RateLimit guidance for SharePoint Online, documenting that SharePoint does not to support this particular feature. There was a preview, but it's no longer there. --------- Co-authored-by: Zain Shahbaz <49994111+zainshahbaz786@users.noreply.github.com> Co-authored-by: James Eccles <jameseccles@users.noreply.github.com> Co-authored-by: Andrew Connell <me@andrewconnell.com> Co-authored-by: pemtaira-msft <151564560+pemtaira-msft@users.noreply.github.com> Co-authored-by: pemtaira <pemtaira+odspmdb@microsoft.com> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Watana <watana.pearb@outlook.com> Co-authored-by: Diego Luces <dluces@users.noreply.github.com> Co-authored-by: dilucesr <dilucesr+odspmdb@microsoft.com> Co-authored-by: TaoziZ03 <106975749+TaoziZ03@users.noreply.github.com> Co-authored-by: titao <titao+odspmdb@microsoft.com> Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com> Co-authored-by: danthemango <danthemango@users.noreply.github.com> Co-authored-by: Steve Pucelik <steve@pucelik.net> Co-authored-by: Jose Barreto <18638509+Jose-Barreto@users.noreply.github.com> Copilot-Session: a7db0d98-be53-48c2-9488-9a226f457144
Summary
Dependency status
maincontaining both stages.Navigation
Before
After
Compatibility
SharePoint/sp-dev-modernizationlinks remain valid through the GitHub redirect topnp/modernization.assessment-tool-classic-pages*articles.Scope
This PR changes documentation navigation, positioning, link routing, labels, legacy notices, and related retirement wording. It doesn't change the Scanner binary, Microsoft 365 Assessment tool, PnP PowerShell, or page transformation behavior.
Current validation
954498fba.docs/toc.ymlas succeeded.ms.dateedits were reverted to theirmainvalues in response to review./azure/active-directory/develop/; the OpenPublishing report no longer references that article.Remaining validation